Skip to content

[202511, Backport #21817] Replace typo dhcpcom with dhcpmon#23349

Merged
vmittal-msft merged 5 commits intosonic-net:202511from
ashutosh-agrawal:backport/21817-to-202511
Mar 31, 2026
Merged

[202511, Backport #21817] Replace typo dhcpcom with dhcpmon#23349
vmittal-msft merged 5 commits intosonic-net:202511from
ashutosh-agrawal:backport/21817-to-202511

Conversation

@ashutosh-agrawal
Copy link
Copy Markdown
Member

@ashutosh-agrawal ashutosh-agrawal commented Mar 26, 2026

Description of PR

Manual port of #21817 to 202511 since cherry-pick failed. This backport is required for PR#23223

Summary:
Fixes # (issue)

Fixes CI errors on PR#23223

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

Required to backport PR for new SONIC DHCv4 relay test scripts to 202511

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Signed-off-by: Ashutosh Agrawal <ashu@cisco.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@ashutosh-agrawal ashutosh-agrawal changed the title [202511] Backport #21817 to 202511 [202511, Backport #21817] Replace typo dhcpcom with dhcpmon Mar 27, 2026
vmittal-msft
vmittal-msft previously approved these changes Mar 27, 2026
@vmittal-msft vmittal-msft enabled auto-merge (squash) March 27, 2026 04:56
@yijingyan2
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Ashutosh Agrawal <ashu@cisco.com>
@ashutosh-agrawal ashutosh-agrawal force-pushed the backport/21817-to-202511 branch from 020cedb to e45e81e Compare March 28, 2026 16:12
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Ashutosh Agrawal <ashu@cisco.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code change LGTM - straightforward typo fix backport. Please fill in the 'Type of change' checkboxes (Bug fix). @ashutosh-agrawal

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ashutosh-agrawal
Copy link
Copy Markdown
Member Author

@vmittal-msft @StormLiangMS PR is ready to be merged.

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Mar 31, 2026

AI agent on behalf of Ying.

  • Unable to verify unresolved review comments; please ensure all human comments are resolved.

@ashutosh-agrawal
Copy link
Copy Markdown
Member Author

Code change LGTM - straightforward typo fix backport. Please fill in the 'Type of change' checkboxes (Bug fix). @ashutosh-agrawal

Done.

@ashutosh-agrawal
Copy link
Copy Markdown
Member Author

AI agent on behalf of Ying.

  • Unable to verify unresolved review comments; please ensure all human comments are resolved.

@yxieca I've replied to the comment. PR is ready to merge.

@vmittal-msft vmittal-msft merged commit c18c3f2 into sonic-net:202511 Mar 31, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants